Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

a few grant admin fixes #8263

Merged
merged 1 commit into from
Jan 19, 2021
Merged

a few grant admin fixes #8263

merged 1 commit into from
Jan 19, 2021

Conversation

owocki
Copy link
Contributor

@owocki owocki commented Jan 18, 2021

Description

this pr does two things

  • makes the grant team member / admin emails easily accessible in the admin, so one can reach out to them easily
  • makes it so the grant admin takes 0.2s to load instead of 5-10s, by adding an optional 'show data' link to the contribution,subscription links in the admin. this is a nice way of offloading prod performance to only use the CPU/DB cycles when needed, and also makes viewing the admins a little nicer UX wise
Refers/Fixes

self

Testing

tested locally

@thelostone-mc thelostone-mc merged commit cb6786c into master Jan 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thelostone-mc thelostone-mc thelostone-mc approved these changes

@amustapha amustapha Awaiting requested review from amustapha

@chibie chibie Awaiting requested review from chibie

@frankchen07 frankchen07 Awaiting requested review from frankchen07

@octavioamu octavioamu Awaiting requested review from octavioamu

@PixelantDesign PixelantDesign Awaiting requested review from PixelantDesign

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@owocki @thelostone-mc